Skip to content

fix: FIT-263: Audio region start and end editor inputs not updating on change #7829

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 24, 2025

Conversation

bmartel
Copy link
Contributor

@bmartel bmartel commented Jun 24, 2025

Kapture.2025-06-24.at.09.57.17.webm

This pull request introduces updates to the RegionEditor.tsx file to enhance type safety and improve state management. The changes include adding a type import for MSTRegion and wrapping a component with MobX's observer to enable reactive rendering.

Type safety improvements:

  • Added the MSTRegion type import to ensure the region prop in RegionEditorProps is strongly typed. ([web/libs/editor/src/components/SidePanels/DetailsPanel/RegionEditor.tsxR22](https://github.com/HumanSignal/label-studio/pull/7829/files#diff-f8ab8d5429cab03058f9ed3cca07385bf31d8044c7798adaeca6667366769468R22))

State management improvements:

  • Wrapped the AudioRegionProperties component with MobX's observer to make it reactive to state changes. ([[1]](https://github.com/HumanSignal/label-studio/pull/7829/files#diff-f8ab8d5429cab03058f9ed3cca07385bf31d8044c7798adaeca6667366769468L86-R87), [[2]](https://github.com/HumanSignal/label-studio/pull/7829/files#diff-f8ab8d5429cab03058f9ed3cca07385bf31d8044c7798adaeca6667366769468L110-R111))

Copy link

netlify bot commented Jun 24, 2025

Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
🔨 Latest commit 8dc0835
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/685abd4d7afb9b00082610b3

Copy link

netlify bot commented Jun 24, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit 8dc0835
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/685abd4d8b59400008273d26

@github-actions github-actions bot added the fix label Jun 24, 2025
Copy link

netlify bot commented Jun 24, 2025

Deploy Preview for label-studio-storybook ready!

Name Link
🔨 Latest commit 8dc0835
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/685abd4d57b6810008cf8b7d
😎 Deploy Preview https://deploy-preview-7829--label-studio-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented Jun 24, 2025

Deploy Preview for label-studio-playground ready!

Name Link
🔨 Latest commit 8dc0835
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-playground/deploys/685abd4dd9be3700080124cb
😎 Deploy Preview https://deploy-preview-7829--label-studio-playground.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@bmartel bmartel requested a review from yyassi-heartex June 24, 2025 15:00
@robot-ci-heartex robot-ci-heartex merged commit 318d812 into develop Jun 24, 2025
44 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-fit-263 branch June 24, 2025 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants